Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgraded sample app #23

Merged
merged 13 commits into from
Aug 26, 2022
Merged

Upgraded sample app #23

merged 13 commits into from
Aug 26, 2022

Conversation

terryyoon-pdftron
Copy link
Contributor

NSDK-168

Overview:

This PR replaces the sample app with a more comprehensive app in order to better demonstrate how the wrapper can be embedded in an existing app and how its API can be used.

The app is a simple file manager within the app's folder with the ability to add files from the device's local storage or from the web, plus other miscellaneous functionalities. It uses this package for opening PDF files and for converting Office files to PDF.

The app is built using Expo SDK 46 with bare workflow (ejected) and React Native version 0.69.4.

Testing:

  • Delete node_modules, and run yarn install
  • Run the app:
    • iOS: yarn ios
    • Android: yarn android

@sgong-pdftron
Copy link
Contributor

@darrenchann could you please perform first round of review, thanks.

@darrenchann
Copy link

darrenchann commented Aug 22, 2022

Tried running project going through repro steps:
(main issue was trying to rebuild the project and that error at the top of error block came up)
image

(tested rerunning project/rebuilding project on master branch with no issues)

@terryyoon-pdftron
Copy link
Contributor Author

@darrenchann is that from this repo or the main React Native repo? the settings.gradle file looks different

@darrenchann
Copy link

@darrenchann is that from this repo or the main React Native repo? the settings.gradle file looks different

Let me review my project structure, it's not looking correct on my end

@darrenchann
Copy link

iOS:
image

android:
image

now able to get both platforms up

Copy link

@darrenchann darrenchann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

was able to run new sample app with both platforms on ios/android on my end

@sgong-pdftron sgong-pdftron removed their request for review August 26, 2022 17:17
@terryyoon-pdftron terryyoon-pdftron merged commit 683ae49 into master Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants